fix(nimbus): make boolean text more visible in dark mode in codemirror windows #13596
Conversation
moibra05
requested review from
freshstrangemusic,
jaredlockhart and
yashikakhurana
as code owners
|
There was a problem hiding this comment.
Thank you @moibra05 this is much readable now, we have codemirror on other pages too we want to make sure we add at all the places like summary page
I don't believe there are any default themes included in the core cm6 library but it is possible to install and integrate third party ones |
|
Yeah I'm just curious if that's a cleaner solution than managing the colours explicitly? @moibra05 @yashikakhurana what do you think? |
|
Incorporating themes would be a more robust solution but based on some research i did, it looks like having it dynamically switch between dark/light modes would add a bit of complexity |
moibra05
force-pushed
the
13594
branch
2 times, most recently
from
8a2a276 to
a76632f
Compare
| basicSetup, | ||
| lineNumbers(), | ||
| drawSelection(), | ||
| highlightSpecialChars(), | ||
| foldGutter(), | ||
| autocompletion(), | ||
| bracketMatching(), | ||
| closeBrackets(), | ||
| highlightSelectionMatches(), | ||
| indentOnInput(), | ||
| highlightActiveLine(), | ||
| syntaxHighlighting(highlightStyle), | ||
| syntaxHighlighting(defaultHighlightStyle), | ||
| history(), | ||
| search(), | ||
| keymap.of([...defaultKeymap, ...searchKeymap, ...historyKeymap]), |
There was a problem hiding this comment.
Yeah this is a bit hairy. If using dedicated light/dark themes is too complicated let's just go back to your original approach of just overriding the colour of the bools. Simplest, smallest solution is usually best 🙏
There was a problem hiding this comment.
The approach i have here is only changing the colour of the boolean text. basicSetup packages all of these extensions without any option to exclude specific ones, which is an issue because this included extension prevents me from setting a custom HighlightStyle that overrides boolean text colour. Annoyingly, the only way to fix this is to manually include all the extensions except defaultHighlightStyle
4 participants
Because
This commit
Fixes #13594